-
Notifications
You must be signed in to change notification settings - Fork 9.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#21786 Fixed asynchronous email sending for the sales entities which were created with disabled email sending #21788
#21786 Fixed asynchronous email sending for the sales entities which were created with disabled email sending #21788
Conversation
…hich were created with disabled email sending
@serhiyzhovnir unfortunately, only members of the maintainers team are allowed to assign developers to the pull request |
Hi @serhiyzhovnir. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @dmytro-ch, thank you for the review. |
Hi @serhiyzhovnir ! I cannot reproduce the issue on 2.3-develop instance. Preconditions:
Steps to reproduce:
Actual Result: Please, check again on the latest 2.3 instance. |
Hi @stoleksiy The main purpose of this pull request is to remove hardcoded values for send email field for sales entities. I've prepared new steps to reproduce. Steps To Reproduce
Trigger Notify ActionsInstall PostmanInstall the Postman https://www.getpostman.com/. Create Magento integration
Trigger Notify Actions for InvoiceThe postman request should look like on the following screenshot: Request URL: {Magento URL}/rest/all/V1/invoices/{Invoice ID}/emails Trigger Notify Actions for ShipmentThe postman request should look like on the following screenshot: Trigger Notify Actions for CreditmemoThe postman request should look like on the following screenshot: Thank you! |
Hi @serhiyzhovnir! Thank you for the detailed description! I will check this PR again! |
✔️ QA Passed |
…-email-sending-for-the-sales-entities-which-were-created-with-disabled-email-sending
Hi @serhiyzhovnir, thank you for your contribution! |
…tities which were created with disabled email sending #21788
Description (*)
Fixed asynchronous email sending for the sales entities which were created with disabled email sending
Fixed Issues (if relevant)
Preconditions (*)
Manual testing scenarios (*)
Contribution checklist (*)